Skip to content

chore(backend): clarify nodemon test ignore configuration (#1081) - #1163

Merged
ogazboiz merged 3 commits into
LabsCrypt:mainfrom
Hikmaholadele:chore/1081-fix-nodemon-test-ignore
Aug 3, 2026
Merged

chore(backend): clarify nodemon test ignore configuration (#1081)#1163
ogazboiz merged 3 commits into
LabsCrypt:mainfrom
Hikmaholadele:chore/1081-fix-nodemon-test-ignore

Conversation

@Hikmaholadele

Copy link
Copy Markdown
Contributor

Closes #1081

Summary

This PR updates the backend nodemon configuration to better reflect the project's current layout and clarifies how test files interact with the development server.

Changes

  • Updated backend/nodemon.json to remove or correct misleading test ignore patterns.
  • Added documentation explaining how nodemon's watch scope relates to the backend/tests directory.
  • Improved configuration clarity for contributors without changing runtime behavior.

Why

The previous configuration included ignore patterns for test files under src, while the project's test suite lives under backend/tests. Since nodemon watches only the configured directories, those ignore patterns were misleading and could cause confusion for contributors.

Testing

  • Verified backend/nodemon.json is valid.
  • Ran the project's relevant validation checks (lint/tests/typecheck as applicable).
  • Confirmed the changes do not alter the development server entrypoint or watch behavior beyond clarifying the configuration.

Closes #1081

@Hikmaholadele
Hikmaholadele force-pushed the chore/1081-fix-nodemon-test-ignore branch from f1ae5a2 to a19557c Compare July 30, 2026 12:43

@ogazboiz ogazboiz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

verified: nodemon watches only src, all tests live in backend/tests, and no test files exist under src, so dropping the ignore patterns is behavior neutral and the README note is right. merging. if you want to keep contributing, join us on Telegram: https://t.me/+DOylgFv1jyJlNzM0

@ogazboiz
ogazboiz merged commit da46cfd into LabsCrypt:main Aug 3, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Infra] backend/nodemon.json ignores *.test.ts but not tests/ directory, so editing a test file still restarts the dev server

2 participants